Skip to content

chore: Remove redundant openai-agents-mcp dependency #134

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Oct 10, 2025
Merged

chore: Remove redundant openai-agents-mcp dependency #134

merged 1 commit into from
Oct 10, 2025

Conversation

aaronsteers
Copy link
Contributor

@aaronsteers aaronsteers commented Oct 10, 2025 •
edited by coderabbitai bot
Loading

chore: Remove redundant openai-agents-mcp dependency

Summary

Removes the openai-agents-mcp>=0.0.8 dependency as it's now redundant. Investigation confirmed that openai-agents 0.3.3 (merged in PR #133) includes native MCP support via the agents.mcp module, making the separate package unnecessary.

Key findings:

  • ✅ Comprehensive codebase search found zero imports of openai-agents-mcp
  • ✅ openai-agents 0.3.3 examples show MCP usage via from agents.mcp import MCPServer, MCPServerStreamableHttp
  • ✅ openai-agents-mcp package last updated March 25, 2025 (no recent development)
  • ✅ Only reference was the dependency declaration itself (now removed)

Review & Testing Checklist for Human

  • Verify MCP functionality works end-to-end by running a connector build that exercises MCP features to ensure no regression
  • Double-check no hidden imports exist by searching for any dynamic imports or indirect usage of openai-agents-mcp that could have been missed

Notes

This change aligns the project with the openai-agents 0.3.3 upgrade that included native MCP support. The root pyproject.toml already has openai-agents-mcp commented out (line 40), indicating this cleanup was anticipated.

Link to Devin run: https://app.devin.ai/sessions/6acb99e3b34a4918af907764ca9b55c3
Requested by: @aaronsteers

Summary by CodeRabbit

  • Chores
    • Removed an unused dependency to streamline installation and reduce potential version conflicts.
    • No changes to features or public APIs; behavior remains the same.
    • Users may experience a smaller install footprint and slightly faster dependency resolution.
    • Improves maintainability by simplifying the dependency graph without affecting configuration or runtime behavior.

@devin-ai-integration @aaronsteers
chore: Remove redundant openai-agents-mcp dependency
e2a490e 
openai-agents 0.3.3 (merged in PR #133) includes native MCP support
via the agents.mcp module. The separate openai-agents-mcp package
is no longer needed. Verified that no code in the codebase imports
or uses openai-agents-mcp.

See examples in openai-agents repo:
https://github.com/openai/openai-agents-python/tree/main/examples/mcp

Co-Authored-By: AJ Steers <[email protected]>
@devin-ai-integration Devin.ai Integration
Copy link
Contributor

Original prompt from AJ Steers 
@Devin - Review this execution log: <https://github.com/airbytehq/connector-builder-mcp/actions/runs/18415068059/job/52476733729?pr=122>

You'll note the "NOW:" "OBSERVED:", "NEXT:" pattern is not being practiced until the work is basically done. This tells me the prompt is not properly routing to the developer instructions in the first step. I also note from that log that the agent is already done but the agent is not aware so it tries to work again on the problem. See if you can find the problem with routing and/or prompting and/or coordination between manager and developer agents.
Thread URL: https://airbytehq-team.slack.com/archives/D089P0UPVT4/p1760121663439179?thread_ts=1760121663.439179

@devin-ai-integration Devin.ai Integration
Copy link
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@github-actions github-actions bot added the chore label Oct 10, 2025
@coderabbitai coderabbitai
Copy link

coderabbitai bot commented Oct 10, 2025 •
edited
Loading

📝 Walkthrough

Walkthrough

Removed the dependency "openai-agents-mcp>=0.0.8" from connector_builder_agents/pyproject.toml. No other files or public interfaces were changed.

Changes

Cohort / File(s) Summary of Changes
Dependency manifest update
connector_builder_agents/pyproject.toml		 Removed dependency entry openai-agents-mcp>=0.0.8; all other dependencies unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

chore

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title clearly and concisely summarizes the pull request’s primary change by stating that the redundant openai-agents-mcp dependency is being removed without including extraneous information or vague terms.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch devin/1760133611-remove-openai-agents-mcp
📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f45ab1a and e2a490e.

⛔ Files ignored due to path filters (1)
  • connector_builder_agents/uv.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • connector_builder_agents/pyproject.toml (0 hunks)
💤 Files with no reviewable changes (1)
  • connector_builder_agents/pyproject.toml
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: Run Evals (Single Connector)
  • GitHub Check: Pytest (Fast)

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions GitHub Actions
Copy link

👋 Greetings, Airbyte Team Member!

Here are some helpful tips and reminders for your convenience.

Testing This Branch via MCP

To test the changes in this specific branch with an MCP client like Claude Desktop, use the following configuration:

{
  "mcpServers": {
    "connector-builder-mcp-dev": {
      "command": "uvx",
      "args": ["--from", "git+https://github.com/airbytehq/connector-builder-mcp.git@devin/1760133611-remove-openai-agents-mcp", "connector-builder-mcp"]
    }
  }
}

Testing This Branch via CLI

You can test this version of the MCP Server using the following CLI snippet:

# Run the CLI from this branch:
uvx 'git+https://github.com/airbytehq/connector-builder-mcp.git@devin/1760133611-remove-openai-agents-mcp#egg=airbyte-connector-builder-mcp' --help

PR Slash Commands

Airbyte Maintainers can execute the following slash commands on your PR:

  • /autofix - Fixes most formatting and linting issues
  • /poe <command> - Runs any poe command in the uv virtual environment
  • /poe build-connector prompt="Star Wars API" - Run the connector builder using the Star Wars API.

📝 Edit this welcome message.

@github-actions GitHub Actions
Copy link

PyTest Results (Fast)

0 tests  ±0   0 ✅ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit e2a490e. ± Comparison against base commit f45ab1a.

@aaronsteers aaronsteers merged commit 1c88a4d into main Oct 10, 2025
15 checks passed
@aaronsteers aaronsteers deleted the devin/1760133611-remove-openai-agents-mcp branch October 10, 2025 22:09
@coderabbitai coderabbitai bot mentioned this pull request Oct 12, 2025
Open
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@aaronsteers aaronsteers

Labels

chore

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@aaronsteers